Skip to content

Update to geth 1.16.5#563

Merged
HerbertJordan merged 2 commits intomainfrom
herbert/update_to_geth_1.16.5
Oct 23, 2025
Merged

Update to geth 1.16.5#563
HerbertJordan merged 2 commits intomainfrom
herbert/update_to_geth_1.16.5

Conversation

@HerbertJordan
Copy link
Copy Markdown
Collaborator

@HerbertJordan HerbertJordan commented Oct 22, 2025 •
edited
Loading

This PR updates dependencies to cover go-ethereum 1.16.5.

The main changes affecting Sonic's code base are:

  • Rename of Message field: the field 'SkipFromEOACheck' got renamed to 'SkipTransactionChecks' in #32641
  • EVM Interface change: Ethereum's StateProcessor now requires a full ChainHeaderReader implementation as an argument #32739
  • StateDB interface change: the introduction of a new tracer argument in the SetCode function of geth's StateDB interface (#32525)
  • some *big.Int got replaced by *uint256.Int

Before this PR can be merged, the following PRs should be merged:

Furthermore, these PRs should be approved:

@HerbertJordan HerbertJordan force-pushed the herbert/update_to_geth_1.16.5 branch 2 times, most recently from 7eab6bf to e9442fb Compare October 22, 2025 16:39
@HerbertJordan HerbertJordan force-pushed the herbert/update_to_geth_1.16.5 branch from e9442fb to 6b4fcfa Compare October 22, 2025 17:29
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 22, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 92.85714% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
opera/contracts/evmwriter/evm_writer.go 33.33% 2 Missing ⚠️
evmcore/tx_pool.go 66.66% 0 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
ethapi/api.go 43.96% <100.00%> (ø)
ethapi/transaction_args.go 85.32% <100.00%> (ø)
evmcore/state_processor.go 97.03% <100.00%> (ø)
evmcore/tx_list.go 96.97% <100.00%> (ø)
gossip/evmstore/carmen.go 77.05% <100.00%> (ø)
gossip/evmstore/statedb_logger.go 40.00% <100.00%> (ø)
gossip/gasprice/reactive.go 94.40% <100.00%> (+0.03%) ⬆️
integration/makegenesis/genesis.go 83.20% <100.00%> (ø)
opera/vm_config.go 93.93% <100.00%> (ø)
utils/int.go 80.95% <100.00%> (+25.39%) ⬆️
... and 2 more

... and 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@HerbertJordan HerbertJordan marked this pull request as ready for review October 22, 2025 21:11
@HerbertJordan HerbertJordan mentioned this pull request Oct 22, 2025
Merged
2 tasks
facuMH
facuMH previously approved these changes Oct 23, 2025
View reviewed changes
@HerbertJordan HerbertJordan requested a review from facuMH October 23, 2025 13:14
LuisPH3
LuisPH3 approved these changes Oct 23, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@LuisPH3 LuisPH3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@HerbertJordan HerbertJordan merged commit d6c534d into main Oct 23, 2025
3 checks passed
@HerbertJordan HerbertJordan deleted the herbert/update_to_geth_1.16.5 branch October 23, 2025 14:14
simonlechner pushed a commit that referenced this pull request Feb 23, 2026
@HerbertJordan @simonlechner
Update to geth 1.16.5 (#563)
696b48d
simonlechner pushed a commit that referenced this pull request Feb 23, 2026
@HerbertJordan @simonlechner
Update to geth 1.16.5 (#563)
b266c61
simonlechner pushed a commit that referenced this pull request Feb 24, 2026
@HerbertJordan @simonlechner
Update to geth 1.16.5 (#563)
9642139
wsodsong pushed a commit that referenced this pull request Feb 25, 2026
@simonlechner @HerbertJordan
Geth dependency upgrade release 2.1.6 (#666)
56146a6 
* Update geth dependency to 1.16.2 (#457)

* Update to geth 1.16.5 (#563)

* Update geth dependency to 1.16.9 (#657)

* Updates for release branch

* Update tosca and carmen to release branches

* Update license in newly added file

---------

Co-authored-by: Herbert <herbert@swrnd.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@facuMH facuMH facuMH approved these changes

@LuisPH3 LuisPH3 LuisPH3 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@HerbertJordan @facuMH @LuisPH3